Skip to content

Add step.sandbox.typescript() - #1784

Merged
jpwilliams merged 25 commits into
jack/sandbox-python-runnerfrom
jack/sandbox-ts-runner
Oct 15, 2026
Merged

Add step.sandbox.typescript()#1784
jpwilliams merged 25 commits into
jack/sandbox-python-runnerfrom
jack/sandbox-ts-runner

Conversation

@jpwilliams

@jpwilliams jpwilliams commented Oct 14, 2026 •
edited
Loading

Copy link
Copy Markdown
Member

Summary

Adds step.sandbox.typescript(), a managed-lifecycle Sandbox that runs TypeScript code and can take input.

Direct step usage

Run TS, with the sandbox's lifecycle managed for you.

inputs become variables accessible by the code.

const stats = await step.sandbox.typescript("sum", `
  return numbers.reduce((a: number, b: number) => a + b, 0)
`, {
  input: { numbers: [1, 2, 3] },
});

Reusable functions

Create a reusable function usable during any run.

Output can also be typed and validated using any StandardSchema-compliant schema library.

import { step } from "inngest";
import { z } from "zod";

export const stats = async (id: string, values: number[]) => {
  return step.sandbox.typescript(id, `
    ...
  `, {
    input: { values },
    schema: { output: z.object({ words: z.number() }),
  });
};
// later in a run
const { words } = await stats("get-stats", event.data.body);

Trace

image image

Checklist

  • Added a docs PR that references this PR N/A Later bump
  • Added unit/integration tests
  • Added changesets if applicable

Related

jpwilliams and others added 9 commits October 13, 2026 23:02
@jpwilliams @claude
feat(sandbox): add sandbox.ts and step.sandbox.ts
064f7bc 
Same shape as the Python runner: input as variables, a top-level return as the
result, and a group of kind ts around create, run and destroy.

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
@jpwilliams @claude
docs(sandbox): name both runners in the runners module comment
6394e82 
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@jpwilliams @claude
Merge jack/sandbox-python-runner into jack/sandbox-ts-runner
c87309f 
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@jpwilliams @claude
Merge jack/sandbox-python-runner (with origin/main) into jack/sandbox…
b43aa43 
…-ts-runner

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@jpwilliams
Merge commit 'd8302d34' into HEAD
d6cb2ee
@jpwilliams @claude
Merge jack/sandbox-python-runner into jack/sandbox-ts-runner
27258fc 
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@jpwilliams @claude
Merge jack/sandbox-python-runner into jack/sandbox-ts-runner
b106838 
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@jpwilliams
Merge remote-tracking branch 'origin/jack/sandbox-python-runner' into…
75173df 
… tmp/rename-ts
@jpwilliams @claude
feat(sandbox): name the TypeScript runner typescript
bcb2d35 
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@jpwilliams jpwilliams self-assigned this Oct 14, 2026
@jpwilliams
jpwilliams added this pull request to stack #1785 October 14, 2026 19:09
@inngest-release-bot inngest-release-bot added the 📦 inngest Affects the `inngest` package label Oct 14, 2026
@greptile-apps

greptile-apps Bot commented Oct 14, 2026 •
edited
Loading

Copy link
Copy Markdown

RetriggerView in GreptileConfidence Score: 4/5

[Medium impact] Adds TypeScript code execution to the sandbox feature.

The advertised output-schema support should be implemented before merging.

Findings

  1. P1 Output schemas are unsupported ▶
Fix with agent prompt 
### Issue 1
packages/inngest/src/components/sandbox/types.ts:569
The new `typescript` method uses `SandboxRunner`, whose options do not accept `schema.output`. Users following the schema example in this PR get a TypeScript error, and the result type cannot be inferred from the schema. If a handle caller bypasses the type check, `schema` reaches the strict command-options check and is rejected before the snippet runs.

Add schema support to the runner options, infer the output type, and check the returned value before resolving.

---

For each issue above, determine whether it is valid and should be fixed. If so, fix it directly.

Summary

Adds step.sandbox.typescript() and sandbox.typescript() using the existing runner helpers.

  • Sandbox callers can now run TypeScript through either runner.

Diagram

 
%%{init: {'theme': 'neutral'}}%%
flowchart TD
  A["step.sandbox.typescript()"] --> B["Create sandbox"]
  B --> C["Run TypeScript command"]
  D["sandbox.typescript()"] --> C
  C --> E["Read returned JSON"]
  E --> F["Return result"]
  B --> G["Destroy after success or final failure"]
Loading

Reviews (1) · Last reviewed commit: "feat(sandbox): name the TypeScript runne..." · Reviewed by Greptile

greptile-apps[bot]
greptile-apps Bot reviewed Oct 14, 2026
View reviewed changes
Comment thread packages/inngest/src/components/sandbox/types.ts Outdated
@changeset-bot

changeset-bot Bot commented Oct 14, 2026 •
edited
Loading

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: 8a44bac

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
inngest Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

jpwilliams and others added 11 commits October 14, 2026 19:33
@jpwilliams @claude
Merge jack/sandbox-python-runner into jack/sandbox-ts-runner
361aa3d 
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@jpwilliams @claude
test(sandbox): add a live suite for the Python and TypeScript runners
8021a52 
Runs every runner case against real Cloud sandboxes through a devstack Dev
Server: `pnpm -C packages/inngest test:sandbox-live`.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@jpwilliams
Merge commit 'b91c1f17' into HEAD
c8e926f
@jpwilliams @claude
test(sandbox): drop the enum from the live TypeScript types case
fd577ac 
Node 26 only strips types, so enums are not supported.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@jpwilliams
Merge remote-tracking branch 'origin/jack/sandbox-python-runner' into…
c9ceb9c 
… HEAD
@jpwilliams @claude
test(sandbox): live case for TypeScript the runner can't strip
13f51ab 
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@jpwilliams @claude
test(sandbox): --keep leaves the live suite's Dev Server running
c0e0c1d 
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@jpwilliams @claude
Merge jack/sandbox-python-runner into jack/sandbox-ts-runner
966ece5 
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@jpwilliams @claude
fix(sandbox): print the Dev Server run URL the UI serves in the live …
093cf87 
…suite

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@jpwilliams
Merge branch 'jack/sandbox-python-runner' into jack/sandbox-ts-runner
c3e6a41
@jpwilliams
Kill live/; internal tool
0fbada6
jpwilliams and others added 2 commits October 15, 2026 13:40
@jpwilliams @claude
Merge origin/jack/sandbox-python-runner into jack/sandbox-ts-runner
d75c583 
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@jpwilliams
Merge remote-tracking branch 'origin/jack/sandbox-python-runner' into…
bc1bc0e 
… HEAD

# Conflicts:
#	packages/inngest/src/components/sandbox/runners.test.ts
@jpwilliams
jpwilliams merged commit f5b7109 into main Oct 15, 2026
46 checks passed
@jpwilliams
jpwilliams deleted the jack/sandbox-ts-runner branch October 15, 2026 18:13
@inngest-release-bot inngest-release-bot mentioned this pull request Oct 15, 2026
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@greptile-apps greptile-apps[bot] greptile-apps[bot] left review comments
@jakobevangelista jakobevangelista jakobevangelista approved these changes
@tonyhb tonyhb Awaiting requested review from tonyhb

Assignees

@jpwilliams jpwilliams

Labels

📦 inngest Affects the `inngest` package

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@jpwilliams @jakobevangelista @inngest-release-bot